Skip to content

Fix calling rez test with empty test name to run all tests#1988

Merged
JeanChristopheMorinPerso merged 6 commits intoAcademySoftwareFoundation:mainfrom
Ni-g-3l:fix/run-all-tests-empty-rez-test-cmd
Jun 21, 2025
Merged

Fix calling rez test with empty test name to run all tests#1988
JeanChristopheMorinPerso merged 6 commits intoAcademySoftwareFoundation:mainfrom
Ni-g-3l:fix/run-all-tests-empty-rez-test-cmd

Conversation

@Ni-g-3l
Copy link
Contributor

@Ni-g-3l Ni-g-3l commented Jun 10, 2025

Fixes #1983

This patch will fix rez test call without any test name.

@Ni-g-3l Ni-g-3l requested a review from a team as a code owner June 10, 2025 18:02
@codecov
Copy link

codecov bot commented Jun 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.05%. Comparing base (d508f0b) to head (8806114).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1988      +/-   ##
==========================================
+ Coverage   60.04%   60.05%   +0.01%     
==========================================
  Files         163      163              
  Lines       20118    20120       +2     
  Branches     3506     3507       +1     
==========================================
+ Hits        12080    12084       +4     
+ Misses       7220     7216       -4     
- Partials      818      820       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Nig3l <nig3lpro@gmail.com>
@Ni-g-3l Ni-g-3l force-pushed the fix/run-all-tests-empty-rez-test-cmd branch from 8e4b397 to 2351808 Compare June 10, 2025 18:12
Nig3l added 2 commits June 10, 2025 21:50
Signed-off-by: Nig3l <nig3lpro@gmail.com>
Signed-off-by: Nig3l <nig3lpro@gmail.com>
Signed-off-by: Jean-Christophe Morin <38703886+JeanChristopheMorinPerso@users.noreply.github.com>
@JeanChristopheMorinPerso JeanChristopheMorinPerso added this to the Next milestone Jun 21, 2025
@JeanChristopheMorinPerso JeanChristopheMorinPerso changed the title Fix calling rez test with empty test name run all tests Fix calling rez test with empty test name to run all tests Jun 21, 2025
Copy link
Member

@JeanChristopheMorinPerso JeanChristopheMorinPerso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Ni-g-3l! Also, thank you for adding a test :)

@JeanChristopheMorinPerso JeanChristopheMorinPerso merged commit d152c9a into AcademySoftwareFoundation:main Jun 21, 2025
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Executing rez test without specifying any test name is not working anymore.

2 participants